Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TextField] Added hintStyle property #1510

Merged
merged 2 commits into from
Sep 29, 2015

Conversation

bgribben
Copy link

Added a 'hintStyle' property to text-field for styling hint text.

Also included documentation.

@hai-cea
Copy link
Member

hai-cea commented Sep 7, 2015

@bgribben bgribben force-pushed the text-field-hint-style-property branch from 2fa6942 to a19f53d Compare September 8, 2015 08:29
@bgribben
Copy link
Author

bgribben commented Sep 8, 2015

Added the prop description @hai-cea - should be good to go now.

shaurya947 added a commit that referenced this pull request Sep 29, 2015
@shaurya947 shaurya947 merged commit 7dad50b into mui:master Sep 29, 2015
@@ -259,7 +261,7 @@ let TextField = React.createClass({
) : null;

let hintTextElement = hintText ? (
<div style={this.mergeAndPrefix(styles.hint)}>{hintText}</div>
<div style={this.mergeAndPrefix(styles.hint, this.props.hintStyle)}>{hintText}</div>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hintStyle instead of this.props.hintStyle

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps trivial but good catch @oliviertassinari

@bgribben quick fix please? :)

@zannager zannager added the component: text field This is the name of the generic UI component, not the React module! label Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: text field This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants